home *** CD-ROM | disk | FTP | other *** search
/ Aminet 7 / Aminet 7 - August 1995.iso / Aminet / comm / net / cslip_sana2.lha / slip / README.Rhialto < prev    next >
Text File  |  1993-07-04  |  4KB  |  84 lines

  1. This is a modified version of Commodores example SANA driver:
  2. the slip.device. As released, it contained a number of bugs and
  3. misfeatures.
  4.  
  5. Additions:
  6. ---------
  7. -   Now also header-compressed slip. (See RFC-1144 by Van Jacobson) This 
  8.     is even compatible with non-comressing slip drivers, as it first 
  9.     tries to trigger the other side of the connection into using 
  10.     compression as well, but ceases these attempts if they appear 
  11.     unsuccessful. In any case, once the other side starts sending 
  12.     compressed headers, it is turned on in the local-to-remote direction 
  13.     as well.
  14.  
  15. Corrected bugs:
  16. --------------
  17. -   S2_GETGLOBALSTATS always returned an error, even though it worked.
  18.     Well, it got the microseconds of the last online wrong.
  19.  
  20. -   S2_TRACKTYPE had two bugs: it would always think a type that you
  21.     requested to track would not yet be tracked, even if it would
  22.     get the packet type right, which it didn't. As a result, every
  23.     call to S2_TRACKTYPE allocated memory for the statistics, yet
  24.     S2_GETTYPESTATS always claimed the type wasn't being tracked.
  25.  
  26. -   S2_GETSTATIONADDRESS should return 2 addresses, the current address,
  27.     and the ROM address. It used to give only a 0.0.0.0 "ROM" address in
  28.     the wrong location. Now, it considers the address in the config
  29.     file the "ROM" address, and the one from S2_CONFIGINTERFACE is
  30.     the current address. Initially both are the same (and both are
  31.     futher ignored since they are meaningless to slip).
  32.  
  33. -   S2_DEVICEQUERY didn't give all required information, and claimed a
  34.     weird size for it.
  35.  
  36. -   Flags set when opening the serial device should be in io->io_SerFlags,
  37.     not in the flags argument to OpenDevice().
  38.  
  39. -   Returned read requests from the serial device were sized by their
  40.     io_Length instead of io_Actual.
  41.  
  42. -   Serial input could potentially overflow the input buffer.
  43.  
  44. -   The code assumed, by using registered arguments, that the callback 
  45.     routines would have their arguments in the correct registers.  Now 
  46.     the type of the pointer explicitly declares the calling convention.
  47.  
  48. -   And the worst of all: it attempted this EXTREMELY STUPID TRICK:
  49.  
  50.     #define SLIPBase ((struct SLIPDevice *)__builtin_getreg(14))
  51.  
  52.     This is terrible!!! That people write code like this! How can you be
  53.     sure the compiler won't use A6 for a register variable somewhere and
  54.     make it unusable in the functions that it calls? In fact, SAS/C 6.2
  55.     does something like that, in ReadConfig..., when it first calls an Exec
  56.     function and then a DOS function, and in both cases fetches the
  57.     required base pointer off A6.
  58.  
  59.     If you want global register variables, use a compiler that supports
  60.     them, such as gcc.
  61.  
  62. Corrected misfeatures:
  63. ---------------------
  64.  
  65. -   The device now runs at the same priority as the opener. This helps
  66.     preventing starvation of the opener in case of high amounts of
  67.     input.
  68.  
  69. -   Uses the serial.device's SERF_EOFMODE mode. This prevents the
  70.     slip.device from hogging the CPU at high priority, and even at equal
  71.     priority improves efficiency.
  72.  
  73. Debatable issues:
  74. ----------------
  75.  
  76. -   Optionally, serial device reads are only initiated when they are
  77.     (already) requested by the caller. This is contrary to the SANA
  78.     philosophy, and therefore makes debugging protocol stacks more
  79.     difficult. On the other hand, why not use the serial device's buffer to
  80.     keep input, instead of dropping it? This also reduces CPU load in case
  81.     the device is open but unused for some reason.  (As it was, the
  82.     slip.device would continually read and decode packets, even if nobody
  83.     wanted them.)
  84.